Nice, I have not tested this, but looks good to me.

I think this works now, can be tried at https://thyttan.github.io/BangleApps/?id=swipeinv

not looking to merge yet though.

@thyttan looks interesting, I'll give it a try when I get a chance, thanks

I think I notice this approach making the watch feel a bit slower than the old one. So that's a minus in that case. But not conclusive, just a feeling.

I also had the idea to do the inversions by wrapping the listener functions at their registration. That way swipeinv would not depend on being first in the listener call chain to work.

It's here: thyttan@5cbb72e

But I didn't get that to work yet.

Doesn't feel too much slower (if at all) now since I added ram keyword to functions.

So you're happy to merge this now?

As it's just an optional app I feel like there's no problem merging this in?

Yes I believe it can go in 🙂

If I find a better approach I can do another PR. But on that note, do you think wrapping listener functions (Bangle.on/prependListener) could be better? I suspect it may not be possible without significantly slowing things down though.

I also had the idea to do the inversions by wrapping the listener functions at their registration. That way swipeinv would not depend on being first in the listener call chain to work.

It's here: thyttan@5cbb72e

But I didn't get that to work yet.

Thanks! I reckon the current method is good enough. Overriding .on/etc feels like it's going to cause issues (removeListener likely won't work then).

Although looking at it, because drag passes an object, you don't need all the re-emit stuff that dragInverter does. Just:

let dragInverter = (e) => {
        "ram"
        const APP_ID = getAppIdFromCurrentFile();
        if (SETTINGS.global ^ Object.keys(SETTINGS.apps).includes(APP_ID)) {
            e.inverted = true;
            if (SETTINGS.global ^ (SETTINGS.apps[APP_ID]&&SETTINGS.apps[APP_ID].dragH)) {e.dx *= -1;}
            if (SETTINGS.global ^ (SETTINGS.apps[APP_ID]&&SETTINGS.apps[APP_ID].dragV)) {e.dy *= -1;}
        }
      }

Because subsequent functions in that chain will get the same (modified) object. That'll speed it up.

swipeInverter needs to be as-is